-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add deployer service #271
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
7fca3ee
to
1377161
Compare
1377161
to
383d9c7
Compare
383d9c7
to
a04b41a
Compare
a04b41a
to
011ac74
Compare
011ac74
to
cbd48ba
Compare
cbd48ba
to
ce94502
Compare
4136506
to
18700bd
Compare
c95520e
to
d073ad6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ctrlc03 thanks, just some comments
packages/coordinator/ts/deployer/__tests__/deployer.service.test.ts
Outdated
Show resolved
Hide resolved
d073ad6
to
8c02006
Compare
8c02006
to
df5c839
Compare
11cbf12
to
029a919
Compare
029a919
to
9138c32
Compare
9138c32
to
ee07b56
Compare
58e8f0d
to
5751832
Compare
95e0cbb
to
343da9f
Compare
cba9d37
to
b19e0d4
Compare
b19e0d4
to
27e50b8
Compare
27e50b8
to
8d06e58
Compare
8d06e58
to
1378034
Compare
// @ts-expect-error - the abi has a more complex type for the processZkeys and tallyZkeys | ||
processZkeys, | ||
// @ts-expect-error - the abi has a more complex type for the processZkeys and tallyZkeys |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will the explicit cast work here? For example: processZkeys: processZkeys as unknown as bigint[]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it wants this type: readonly { alpha1: { x: bigint; y: bigint; }; beta2: { x: readonly [bigint, bigint]; y: readonly [bigint, bigint]; }; gamma2: { x: readonly [bigint, bigint]; y: readonly [bigint, bigint]; }; delta2: { x: readonly [bigint, bigint]; y: readonly [bigint, bigint]; }; ic: readonly { ...; }[]; }[]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, maybe it can be cast to G2Point[]?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that wasn't working either
// @ts-expect-error - the abi has a more complex type for the processZkeys and tallyZkeys | ||
processZkeys, | ||
// @ts-expect-error - the abi has a more complex type for the processZkeys and tallyZkeys |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's try cast here too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot! it's a quite large PR
1378034
to
5dbceb9
Compare
5dbceb9
to
0c7dc30
Compare
Implement a Deployer service on the coordinator service
Note: seems like when deploying the poseidon libraries they don't work, so for now I would just use the already deployed ones which we have on all chains. End of the day we should encourage re-use of those.
fix #61
fix #267